CAT-945 - Update README.md LICENSE #1408
Conversation
Ramesh7
requested review from
alexjfisher,
b4ldr,
bastelfreak,
ekohl,
smortex,
seanmil and
a team
as code owners
README.md
Outdated
| @@ -577,6 +578,10 @@ As of Puppet Enterprise 3.7, the stdlib module is no longer included in PE. PE u | |||
|
|
|||
| For an extensive list of supported operating systems, see [metadata.json](https://github.com/puppetlabs/puppetlabs-stdlib/blob/main/metadata.json) | |||
|
|
|||
| ## License | |||
|
|
|||
| This codebase is licensed under the Apache2.0 licensing, however due to the nature of the codebase the open source dependencies may also use a combination of [AGPL](https://www.gnu.org/licenses/agpl-3.0.en.html), [GPL2.0](https://www.gnu.org/licenses/old-licenses/gpl-2.0.en.html) and [MIT](https://opensource.org/license/mit/) Licensing. | |||
There was a problem hiding this comment.
Is there a reason you restrict it to AGPL, GPL2 and MIT?
There was a problem hiding this comment.
the license list isn't complete, if you do a bundle install and then:
Gem.loaded_specs.each do |name, spec|
puts "#{name}: #{spec.license}"
end
ansi: BSD-2-Clause
mutex_m: Ruby
pluginator: LGPL v3
There was a problem hiding this comment.
This change is to specifically meet some license scanning requirements in our internal tooling which wants us to make sure for those particular licenses we declare. Note we say may as were making a general statement across tools and modules
There was a problem hiding this comment.
Thanks @davidsandilands for putting background here and Thanks @bastelfreak for nice tip.
There was a problem hiding this comment.
Added missing one except Ruby one, not sure if we need to explicitly need to mention here, @davidsandilands please review and confirm the same. Thanks
Ramesh7
force-pushed
the
CAT-945-license-update
branch
from
5b79539 to
99fece0
Compare
Issue
Summary
Update README.md with declaring license details.
Checklist
puppet apply)